Skip to content

[wip] electron investigations #3956

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 44 commits into
base: main
Choose a base branch
from
Draft

[wip] electron investigations #3956

wants to merge 44 commits into from
+2,117 −24

Conversation

@thomas-lebeau
Copy link
Collaborator

@thomas-lebeau thomas-lebeau commented Nov 11, 2025

Motivation

Changes

Test instructions

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@cit-pr-commenter
Copy link

cit-pr-commenter bot commented Nov 11, 2025 •
edited
Loading

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 164.05 KiB 164.16 KiB +109 B +0.06% ✅
Rum Profiler 4.84 KiB 4.84 KiB 0 B 0.00% ✅
Rum Recorder 19.50 KiB 19.50 KiB 0 B 0.00% ✅
Logs 55.91 KiB 55.98 KiB +76 B +0.13% ✅
Flagging 944 B 944 B 0 B 0.00% ✅
Rum Slim 121.33 KiB 121.40 KiB +76 B +0.06% ✅
Worker 23.63 KiB 23.63 KiB 0 B 0.00% ✅
🚀 CPU Performance
Action Name Base CPU Time (ms) Local CPU Time (ms) 𝚫%
RUM - add global context 0.0038 0.006 +57.89%
RUM - add action 0.0109 0.0159 +45.87%
RUM - add error 0.0105 0.0173 +64.76%
RUM - add timing 0.0025 0.0048 +92.00%
RUM - start view 0.003 0.0057 +90.00%
RUM - start/stop session replay recording 0.0006 0.0012 +100.00%
Logs - log message 0.0122 0.0237 +94.26%
🧠 Memory Performance
Action Name Base Memory Consumption Local Memory Consumption 𝚫
RUM - add global context 25.75 KiB 26.05 KiB +308 B
RUM - add action 96.59 KiB 48.53 KiB -48.06 KiB
RUM - add timing 24.39 KiB 24.10 KiB -305 B
RUM - add error 82.49 KiB 53.77 KiB -28.73 KiB
RUM - start/stop session replay recording 23.62 KiB 23.21 KiB -419 B
RUM - start view 439.69 KiB 424.61 KiB -15.08 KiB
Logs - log message 44.42 KiB 45.02 KiB +622 B

🔗 RealWorld

@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Nov 11, 2025 •
edited by datadog-official bot
Loading

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
• Patch Coverage: 75.00%
• Total Coverage: 92.67% (+0.00%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 73d283e | Docs | Datadog PR Page | Was this helpful? Give us feedback!

thomas-lebeau and others added 19 commits November 11, 2025 10:36
@thomas-lebeau
Merge branch 'benoit/reduce-build-boilerplate' into thomas.lebeau/ele…
540c639 
…ctron
@thomas-lebeau
Merge branch 'benoit/reduce-build-boilerplate' into thomas.lebeau/ele…
d81ce59 
…ctron
@thomas-lebeau
🙈 [wip] Add Electron package with initial setup and integration for D…
bbcfe4e 
…atadog tracing
@bcaudan
Extract sendRumEvent + Add ApplicationLaunch view
a16f3a6
@bcaudan
add dummy exception tracking to link with view / session context
ac4259c
@bcaudan
Track activity and update view time spent
e7fd56a
@bcaudan
Use observable to send and assemble RUM events
154ad4e
@bcaudan
Generate RUM errors from error spans
949f563
@thomas-lebeau
♻️ rename bride.ts to renderer and move expoerted function in separat…
0ef6e03 
…e files
@thomas-lebeau
✨ add ipcMain and move tracer
19dfc92
@thomas-lebeau
refactor: move setupMainBridge in it's own file
86ff2f3
@mormubis
🚧 send trace
7134ff5
@mormubis
♻️ put back batching
1678750
@thomas-lebeau
refactor: better ipc span name
a514452
@bcaudan
attach RUM context to spans
08d9503
@bcaudan
extract rum files
200a387
@bcaudan
extract trace directory
8562721
@thomas-lebeau
feat: add electron plugin for browser-rum
dbaeeff
@thomas-lebeau
fix: conflicts
11e5c05
@bcaudan bcaudan force-pushed the thomas.lebeau/electron branch from 79884fc to 9b78148 Compare November 18, 2025 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@thomas-lebeau @BenoitZugmeyer @bcaudan @mormubis